Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Issue #202 -Ensure Protection to all the routes of the website such about us and discussion #238

Merged
merged 4 commits into from
Oct 25, 2024
Merged

Issue #202 -Ensure Protection to all the routes of the website such about us and discussion #238

merged 4 commits into from
Oct 25, 2024

Conversation

2004vivek
Copy link
Contributor

Title and Issue number

Title:
Ensure Protection to all the routes of the website such about us and discussion

Issue No.:
Issue #202

Close #202
I have resolved the issue of the navigating to the protected route in such a way that if the user is not logged in then they will not be able to access the protected routes

Screenshots/Video (mandatory)

welcome.mp4

Checklist:

  • [X ] I have mentioned the issue number in my Pull Request.
  • [X ] I have commented my code, particularly in hard-to-understand areas.
  • [X ] I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.
  • [X ] I have gone through the contributing.md file before contributing.

Additional context (Mandatory):
I have fixed the issue of accessing the protected route

Are you contributing under any Open-source programme?

  • I'm a GSSOC-EXT contributor
  • I'm a HACKTOBERFEST contributor

@2004vivek
Update App.jsx
e46ed0d 
Signed-off-by: 2004vivek <[email protected]>
@2004vivek
Create ProtectedRoute.jsx
dce509c 
Signed-off-by: 2004vivek <[email protected]>
@2004vivek
Update App.jsx
5a8757b 
Signed-off-by: 2004vivek <[email protected]>
@2004vivek
Update Login.jsx
1bd702c 
Signed-off-by: 2004vivek <[email protected]>
@netlify Netlify
Copy link

netlify bot commented Oct 24, 2024 •
edited
Loading

👷 Deploy request for bitbox-in accepted.

Name Link
🔨 Latest commit 1bd702c
🔍 Latest deploy log https://app.netlify.com/sites/bitbox-in/deploys/671a5bd516e8fb0008b83b06

@github-actions GitHub Actions
Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our chaotic CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@Anuj3553 Anuj3553 self-requested a review October 25, 2024 16:32
@Anuj3553 Anuj3553 added gssoc-ext Extended contribution for GSSoC hacktoberfest-accepted Eligible contribution for Hacktoberfest rewards level2 Intermediate-level contribution or task hacktoberfest Open-source contribution during October for GSSoC labels Oct 25, 2024
@Anuj3553 Anuj3553 merged commit 6feebdc into Bitbox-Connect:main Oct 25, 2024
1 of 2 checks passed
@Anuj3553
Copy link
Collaborator

Well done @2004vivek

@github-actions GitHub Actions
Copy link

🎉🎉 Thank you for your contribution! Your PR #238 has been merged! 🎉🎉

@Anuj3553
Copy link
Collaborator

Can you give star to this repo?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Anuj3553 Anuj3553 Awaiting requested review from Anuj3553

Assignees

@2004vivek 2004vivek

Labels
gssoc-ext Extended contribution for GSSoC hacktoberfest Open-source contribution during October for GSSoC hacktoberfest-accepted Eligible contribution for Hacktoberfest rewards level2 Intermediate-level contribution or task
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature Request]: Ensure Protection to all the routes of the website such about us and discussion
2 participants
@2004vivek @Anuj3553